Skip to content

Add new setting AllowConnFromSubnets#335

Open
ijc wants to merge 1 commit into
MythTV:masterfrom
ijc:allow-conn-from-subnet
Open

Add new setting AllowConnFromSubnets#335
ijc wants to merge 1 commit into
MythTV:masterfrom
ijc:allow-conn-from-subnet

Conversation

@ijc

@ijc ijc commented Mar 14, 2021

Copy link
Copy Markdown
Contributor

This is a fine grained version of AllowConnFromAll which allows configuration
of specific subnets or hosts.

Checklist

This is a fine grained version of `AllowConnFromAll` which allows configuration
of specific subnets or hosts.
@ijc

ijc commented Mar 14, 2021

Copy link
Copy Markdown
Contributor Author

I was unsure where to document this, I don't find the existing AllowConnFromAll anywhere in the wiki and the google hits are all the code & the original trac ticket https://code.mythtv.org/trac/ticket/13024 that added it.

@bennettpeter

Copy link
Copy Markdown
Member

@ijc

ijc commented Mar 14, 2021

Copy link
Copy Markdown
Contributor Author

@bennettpeter thanks, I failed to find it because it doesn't mention the db name, which now I think of it makes perfect sense since that's really an internal detail.

I'm just about to turn in for the night so I'll propose some words for the wiki here later.

@garybuhrmaster

Copy link
Copy Markdown
Contributor

As I recall, QHostAddress::parseSubnet did not support IPv6 addresses. Has that been fixed in Qt?

@garybuhrmaster

Copy link
Copy Markdown
Contributor

As I recall, QHostAddress::parseSubnet did not support IPv6 addresses. Has that been fixed in Qt?

Never mind, I found my own answer, it has been fixed for IPv6 (later than I would have liked, which should have happened over a decade ago).

@stuarta

stuarta commented Mar 15, 2021

Copy link
Copy Markdown
Member

@ijc Can you raise an issue describing what the issue is that this is resolving please?

@ijc

ijc commented Mar 15, 2021

Copy link
Copy Markdown
Contributor Author

@stuarta done, please see #339.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

More fine grained control over client IPs allowed to connect to backend

4 participants